Fix R__zip function for ROOT 6.34 #1526
Conversation
jokonig
commented
Oct 20, 2025
jokonig
commented
- With ROOT v3.34, the functionality R__zipMultipleAlgorithm is not yet available, however R__zip also does not seem available.
- A dummy function for R__zip is added in case the ROOT version is between 6.28 and 6.38. This means that the functionality of this function is lost in that cases.
- Tested locally: AliRoot + AliPhysics build with the latest updated alidist + local train-tests were sucessful
- This modifies what was implemented in: Add support for ROOT v6.36.00 #1525
- Might not be the best solution! Please check!
- With ROOT v3.34, the functionality R__zipMultipleAlgorithm is not yet available, however R__zip also does not seem available. - A dummy function for R__zip is added in case the ROOT version is between 6.28 and 6.38. This means that the functionality of this function is lost in that cases.
|
c550533: approval required: 1 of @chiarazampolli (Chiara Zampolli), @davidrohr (David Rohr), @heikoengel (Heiko Engel), @jgrosseo (Jan Fiete Grosse-Oetringhaus), @ktf (Giulio Eulisse), @mkrzewic (Mikolaj Krzewicki), @pzhristov (Peter Hristov), @qgp (Jochen Klein), @sawenzel (Sandro Christian Wenzel), @sgorbuno (Sergey Gorbunov), @shahor02 (Ruben Shahoyan) Comment with |
|
Can't you simply update to 6.36? it's now in production. |
|
@ktf when you say its in production, does that mean these PRs (like alisw/AliPhysics#24074) will also go through automatically? |
|
@ktf With the ROOT version 6.36 it still breaks. Looking at the open PRs, the log says "-- Found ROOT version 6.36.04" |
4 participants
